Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update client.py #934

Merged
merged 1 commit into from
Dec 18, 2023
Merged

Update client.py #934

merged 1 commit into from
Dec 18, 2023

Conversation

eltociear
Copy link
Collaborator

Why are these changes needed?

overriden -> overridden

Checks

overriden -> overridden
@codecov-commenter
Copy link

codecov-commenter commented Dec 10, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9cec541) 26.63% compared to head (7559bdc) 37.96%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #934       +/-   ##
===========================================
+ Coverage   26.63%   37.96%   +11.33%     
===========================================
  Files          28       28               
  Lines        3777     3777               
  Branches      858      900       +42     
===========================================
+ Hits         1006     1434      +428     
+ Misses       2700     2217      -483     
- Partials       71      126       +55     
Flag Coverage Δ
unittests 37.91% <ø> (+11.33%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@qingyun-wu qingyun-wu added this pull request to the merge queue Dec 18, 2023
Merged via the queue into microsoft:main with commit 7dd88b6 Dec 18, 2023
76 of 84 checks passed
rlam3 pushed a commit to rlam3/autogen that referenced this pull request Dec 19, 2023
overriden -> overridden
whiskyboy pushed a commit to whiskyboy/autogen that referenced this pull request Apr 17, 2024
…rs (microsoft#934)

* add basic support to Spark dataframe

add support to SynapseML LightGBM model

update to pyspark>=3.2.0 to leverage pandas_on_Spark API

* clean code, add TODOs

* add sample_train_data for pyspark.pandas dataframe, fix bugs

* improve some functions, fix bugs

* fix dict change size during iteration

* update model predict

* update LightGBM model, update test

* update SynapseML LightGBM params

* update synapseML and tests

* update TODOs

* Added support to roc_auc for spark models

* Added support to score of spark estimator

* Added test for automl score of spark estimator

* Added cv support to pyspark.pandas dataframe

* Update test, fix bugs

* Added tests

* Updated docs, tests, added a notebook

* Fix bugs in non-spark env

* Fix bugs and improve tests

* Fix uninstall pyspark

* Fix tests error

* Fix java.lang.OutOfMemoryError: Java heap space

* Fix test_performance

* Update test_sparkml to test_0sparkml to use the expected spark conf

* Remove unnecessary widgets in notebook

* Fix iloc java.lang.StackOverflowError

* fix pre-commit

* Added params check for spark dataframes

* Refactor code for train_test_split to a function

* Update train_test_split_pyspark

* Refactor if-else, remove unnecessary code

* Remove y from predict, remove mem control from n_iter compute

* Update workflow

* Improve _split_pyspark

* Fix test failure of too short training time

* Fix typos, improve docstrings

* Fix index errors of pandas_on_spark, add spark loss metric

* Fix typo of ndcgAtK

* Update NDCG metrics and tests

* Remove unuseful logger

* Use cache and count to ensure consistent indexes

* refactor for merge maain

* fix errors of refactor

* Updated SparkLightGBMEstimator and cache

* Updated config2params

* Remove unused import

* Fix unknown parameters

* Update default_estimator_list

* Add unit tests for spark metrics
whiskyboy pushed a commit to whiskyboy/autogen that referenced this pull request Apr 17, 2024
overriden -> overridden
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants